diff mbox series

[1/2] Makefile: Remove -mno-direct-move cflag

Message ID 20180723233722.9252-2-joel@jms.id.au
State Accepted
Headers show
Series makefile: tweak some flags | expand

Checks

Context Check Description
snowpatch_ozlabs/apply_patch success master/apply_patch Successfully applied

Commit Message

Joel Stanley July 23, 2018, 11:37 p.m. UTC 
GCC 8 warns that -mno-direct-move is depreciated. We had it there so
we wouldn't use VSX registers in skiboot, as they are not
saved/restored, however Segher confirms:

 > if you already have -mno-altivec  then -mno-direct-move does zilch

So it was never doing anything.

Resolves: open-power/skiboot#186
Signed-off-by: Joel Stanley <joel@jms.id.au>
---
 Makefile.main | 1 -
 1 file changed, 1 deletion(-)
diff mbox series

Patch

diff --git a/Makefile.main b/Makefile.main
index a155e1e66601..487aa735a553 100644
--- a/Makefile.main
+++ b/Makefile.main
@@ -179,7 +179,6 @@  CFLAGS += $(call try-cflag,$(CC),-mno-multiple)
 # do not use any automatic vector foo
 # While it would be safe during boot, we don't save/restore across OPAL calls
 CFLAGS += $(call try-cflag,$(CC),-mno-vsx) \
-	$(call try-cflag,$(CC),-mno-direct-move) \
 	$(call try-cflag,$(CC),-mno-altivec)
 
 # Do not use load/store update. You REALLY do not want to use this!