[patch #9442] Add flang (LLVM-based compiler) support

2024-01-31 Thread INVALID.NOREPLY
Follow-up Comment #3, patch#9442 (group libtool):

Seems this patch does not fully address the problem.

+ libtool --version
 libtool (GNU libtool) 2.4.7.65-e11f

For example, building hdf5 with the ARM compiler, with an MPI compiler
wrapper
I still get this error:

flang: error: unknown argument: '-soname'

Looking at the libtool script produced by configure,
I still see empty wl= 

11810 # ### BEGIN LIBTOOL TAG CONFIG: FC
11811   
11812 # The linker used to build libraries.
11813
LD="/bm/ashterenli/install/arm-compiler-for-linux_23.10_AmazonLinux-2/gcc-12.2.0_AmazonLinux-2/aarch64-linux-gnu/bin/ld"
11814 
11815 # How to create reloadable object files.
11816 reload_flag=" -r"
11817 reload_cmds="\$LD\$reload_flag -o \$output\$reload_objs"
11818 
11819 # Commands used to build an old-style archive.
11820 old_archive_cmds="\$AR \$AR_FLAGS \$oldlib\$oldobjs~\$RANLIB
\$tool_oldlib"
11821 
11822 # A language specific compiler.
11823 CC="ftn"
11824 
11825 # Is the compiler the GNU compiler?
11826 with_gcc=no
11827   
11828 # Compiler flag to turn off builtin functions.
11829 no_builtin_flag=""
11830   
11831 # Additional compiler flags for building library objects.
11832 pic_flag=""
11833   
11834 # How to pass a linker flag through the compiler.
11835 wl=""   


And I still have to fix the libtool script with:

sed -i -e 's/wl=""/wl="-Wl,"/g' libtool

which I got from: https://github.com/Unidata/netcdf-fortran/issues/309






___

Reply to this item at:

  

___
Message sent via Savannah
https://savannah.gnu.org/




Re: [patch #9442] Add flang (LLVM-based compiler) support

2024-01-16 Thread Roumen Petrov

Hi,


Mike Frysinger wrote:

Update of patch#9442 (group libtool):

   Status:None => Done
  Open/Closed:Open => Closed

 ___

Follow-up Comment #1:

i dropped the -DPIC part as no other target does it, and i think we handle
that in common code already.  feel free to follow up if you disagree.


Please note that by default libtool build shared and static objects .
When is build shared object -DPIC is set internally for shared build.
So to remove -DPIC looks harmless.

I guess that reporter could check this. See libtool manual:
...

Notice that the second run of GCC has its output discarded. This is done so 
that compiler warnings aren’t annoyingly duplicated. If you need to see both 
sets of warnings (you might have conditional code inside ‘#ifdef PIC’ for 
example), you can turn off suppression with the -no-suppress option to 
libtool’s compile mode:

burger$libtool --mode=compile gcc -no-suppress -g -O -c hello.c
gcc -g -O -c hello.c  -fPIC -DPIC -o .libs/hello.o
gcc -g -O -c hello.c -o hello.o
burger$





Regards,
Roumen Petrov



[patch #9442] Add flang (LLVM-based compiler) support

2024-01-13 Thread Brad Smith
Follow-up Comment #2, patch#9442 (group libtool):

This should be checking for *flang* to also match for the current name of
flang-new, though hopefully for LLVM 18 that'll change to flang, but not yet.


___

Reply to this item at:

  

___
Message sent via Savannah
https://savannah.gnu.org/




[patch #9442] Add flang (LLVM-based compiler) support

2024-01-13 Thread Mike Frysinger
Update of patch#9442 (group libtool):

  Status:None => Done   
 Open/Closed:Open => Closed 

___

Follow-up Comment #1:

i dropped the -DPIC part as no other target does it, and i think we handle
that in common code already.  feel free to follow up if you disagree.

https://git.savannah.gnu.org/cgit/libtool.git/commit/?h=33d4b69fccbbb557c1160cc46c3d8d2f30fb8a2d


___

Reply to this item at:

  

___
Message sent via Savannah
https://savannah.gnu.org/




[patch #9442] Add flang (LLVM-based compiler) support

2017-08-24 Thread Gilles Gouaillardet
URL:
  

 Summary: Add flang (LLVM-based compiler) support
 Project: GNU Libtool
Submitted by: ggouaillardet
Submitted on: Thu 24 Aug 2017 08:29:22 AM UTC
Category: None
Priority: 5 - Normal
  Status: None
 Privacy: Public
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any

___

Details:

This patch adds support for flang compilers. Some specific flags are needed so
these compilers must be handled in a specific way.
By default, the compiler is called 'flang', but ARM releases their own
flang-based compiler called 'armflang'.

Because of the current lack of flang support in libtool, the generated
'libtool' must be manually modified after 'configure' is invoked. Such a
process is for example described on ARM web site (it involves the Open MPI
library) at
https://developer.arm.com/products/software-development-tools/hpc/resources/porting-and-tuning/building-openmpi-with-arm-compiler



___

File Attachments:


---
Date: Thu 24 Aug 2017 08:29:22 AM UTC  Name: flang.patch  Size: 890B   By:
ggouaillardet



___

Reply to this item at:

  

___
  Message sent via/by Savannah
  http://savannah.gnu.org/