[llvm] [BPF] Use ".L" local prefix label for basic blocks (PR #95103)

via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 24 12:05:01 PDT 2024


4ast wrote:

> Okay, looks like consensus is to use '.L' prefix for private global prefix and private label prefix. @aengelke could you make the change again? The below should work:
> 
> ```
> diff --git a/llvm/lib/Target/BPF/MCTargetDesc/BPFMCAsmInfo.h b/llvm/lib/Target/BPF/MCTargetDesc/BPFMCAsmInfo.h
> index 7b2168458c93..1173fe2f7590 100644
> --- a/llvm/lib/Target/BPF/MCTargetDesc/BPFMCAsmInfo.h
> +++ b/llvm/lib/Target/BPF/MCTargetDesc/BPFMCAsmInfo.h
> @@ -25,6 +25,7 @@ public:
>        IsLittleEndian = false;
>  
>      PrivateGlobalPrefix = ".L";
> +    PrivateLabelPrefix = ".L";
>      WeakRefDirective = "\t.weak\t";
> ```

No. The current diff changed things to work properly with "L".

https://github.com/llvm/llvm-project/pull/95103


More information about the llvm-commits mailing list