[llvm] r268155 - Differential Revision: http://reviews.llvm.org/D19753

[David Blaikie via llvm-commits]

On Fri, Apr 29, 2016 at 9:18 PM, Sriraman Tallam via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> Author: tmsriram
> Date: Fri Apr 29 23:18:52 2016
> New Revision: 268155
>
> URL: http://llvm.org/viewvc/llvm-project?rev=268155&view=rev
> Log:
> Differential Revision: http://reviews.llvm.org/D19753
>
> Delete Target Option PositionIndependentExecutable as PIE is now part of
> module flags.
>

Note for next time: check the commit message before it goes in, Phab
doesn't always do the greatest stuff here.

The descriptive text should be on the first line (as it becomes the commit
summary in version control logs, and becomes the subject line in the commit
email) - usually the Differential Revision line is the last one.


>
> Modified:
>     llvm/trunk/include/llvm/Target/TargetOptions.h
>
> Modified: llvm/trunk/include/llvm/Target/TargetOptions.h
> URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Target/TargetOptions.h?rev=268155&r1=268154&r2=268155&view=diff
>
> ==============================================================================
> --- llvm/trunk/include/llvm/Target/TargetOptions.h (original)
> +++ llvm/trunk/include/llvm/Target/TargetOptions.h Fri Apr 29 23:18:52 2016
> @@ -96,8 +96,7 @@ namespace llvm {
>            UnsafeFPMath(false), NoInfsFPMath(false), NoNaNsFPMath(false),
>            HonorSignDependentRoundingFPMathOption(false),
> NoZerosInBSS(false),
>            GuaranteedTailCallOpt(false), StackAlignmentOverride(0),
> -          StackSymbolOrdering(true),
> -          EnableFastISel(false), PositionIndependentExecutable(false),
> +          StackSymbolOrdering(true), EnableFastISel(false),
>            UseInitArray(false), DisableIntegratedAS(false),
>            CompressDebugSections(false), FunctionSections(false),
>            DataSections(false), UniqueSectionNames(true),
> TrapUnreachable(false),
> @@ -180,12 +179,6 @@ namespace llvm {
>      /// compile time.
>      unsigned EnableFastISel : 1;
>
> -    /// PositionIndependentExecutable - This flag indicates whether the
> code
> -    /// will eventually be linked into a single executable, despite the
> PIC
> -    /// relocation model being in use. It's value is undefined (and
> irrelevant)
> -    /// if the relocation model is anything other than PIC.
> -    unsigned PositionIndependentExecutable : 1;
> -
>      /// UseInitArray - Use .init_array instead of .ctors for static
>      /// constructors.
>      unsigned UseInitArray : 1;
> @@ -273,7 +266,6 @@ inline bool operator==(const TargetOptio
>      ARE_EQUAL(GuaranteedTailCallOpt) &&
>      ARE_EQUAL(StackAlignmentOverride) &&
>      ARE_EQUAL(EnableFastISel) &&
> -    ARE_EQUAL(PositionIndependentExecutable) &&
>      ARE_EQUAL(UseInitArray) &&
>      ARE_EQUAL(TrapUnreachable) &&
>      ARE_EQUAL(EmulatedTLS) &&
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160430/6b39f787/attachment.html>