On Tue, Dec 22, 2015 at 2:30 PM, Peter Bergner <berg...@vnet.ibm.com> wrote: > On Tue, 2015-12-22 at 13:36 -0500, David Edelsohn wrote: >> On Tue, Dec 22, 2015 at 12:13 PM, Peter Bergner <berg...@vnet.ibm.com> wrote:
>> Ultimately rs6000_file_start ".machine" directive support should be >> strengthened to handle -mcpu=. > > I'm not sure there is anything to do there. The code that emits the > .machine assembler directive uses rs6000_isa_flags which should be > set correctly, regardless of whether we use -mcpu= with power8 or > powerpc64le. ...although, the code as is never outputs a .machine > on LE, since the code that guards the emitting of .machine doesn't > ever seem to be true on LE. At least I can't seem to get a .machine > generated with my LE build. Currently, the .machine directive only is emitted if -mcpu= is not used. We should avoid -mcpu= setting assembler command line options and instead switch to use .machine. The .machine path needs to be tested more thoroughly in the presence of -mcpu options. Thanks, David